PR을 main으로 잘못 날려서, 기존 PR을 닫고 다시 작성합니다! 😭

체크 리스트

• 미션의 필수 요구사항을 모두 구현했나요?
• Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
• 애플리케이션이 정상적으로 실행되나요?

어떤 부분에 집중하여 리뷰해야 할까요?

1. 기존 BlackjackGame.start() 구조를 유지하기 위해 생성 정보 객체를 추가한 선택이 적절한가?

기존 구조에서는 BlackjackGame.start()에 플레이어 이름 목록(List<String>)을 전달하고,
BlackjackGame 내부에서 이를 바탕으로 Player를 생성하는 방식으로 구현되어 있었습니다.

이번 사이클에서 베팅 기능이 추가되면서, 더 이상 이름만으로는 Player를 생성할 수 없게 되었고,
이름과 함께 BettingMoney도 초기화해야 하는 상황이 생겼습니다.

이때 가장 먼저 고민했던 부분은, 기존의 구조를 최대한 유지하면서 확장하자는 것이었습니다.
기존처럼 BlackjackGame.start()가 게임 시작과 플레이어 생성을 함께 담당하는 구조를 유지하도록 설계하고 싶었고,
그래서 start()의 역할은 유지하되, 단순 이름 목록 대신 이름과 배팅 금액을 함께 담는 PlayerCreationInfo 객체를 만들고, 그 리스트를 넘겨주는 방식을 선택했습니다.

이 방식은 BlackjackGame.start() 가 여전히 게임 생성 책임을 가지도록 유지할 수 있고,
GameController가 도메인 객체 생성에 깊게 관여하지 않도록 할 수 있다는 점에서 괜찮다고 판단했습니다.

다만 PlayerCreationInfo는 실제 게임 진행 중에는 사용되지 않고 생성 시점에만 사용되기 때문에,
이 객체가 도메인적으로 의미가 있는 객체인지,
아니면 기존 구조를 유지하기 위해 만든 단순 포장 객체에 가까운지에 대해 고민이 있었습니다.

현재 방식이 적절한 선택인지,
혹은 플레이어 생성 책임을 다른 객체로 분리하는 것이 더 자연스러운지
제이미의 의견을 듣고 싶습니다.

2. Hand의 책임 분리

1. Score가 Hand를 만들기
2. Hand가 Score를 만들기
3. 제 3자가 Score를 만들기
   이렇게 적었어야 했는데 오타가 났네요.
   제 3자가 Score를 만들고 있지만 결국 생성에 대한 행위는 Hand가 하고 있는 상황인데요.
   뭔가, 3번을 의도했지만 어느정도 2번에 걸쳐있는 것 같아 책임 분리가 덜 되어보이기도 하는데, 의도된 부분인지 궁금했어요.

    public Score getScore() {
        return new Score(ScoreCalculator.calculate(List.copyOf(cards)));
    }

Score를 Hand의 상태로 보는 것이 자연스럽다고 생각해서 Hand.getScore()를 두었고, 다만 점수 계산 규칙까지 Hand가 모두 알 필요는 없다고 판단해 계산 과정은 ScoreCalculator에 위임했습니다. 그래서 현재는 Hand가 계산 결과를 받아 Score를 생성해 반환하는 구조로 두었습니다.

다만 리뷰 주신 내용을 보면서, 이 구조가 완전히 “제3자가 Score를 만든다”기보다는 Hand가 생성 책임까지 일부 가지고 있어서 2번과 3번 사이에 걸쳐 있는 형태로도 보일 수 있겠다고 느꼈습니다.

여기서 궁금한 부분이 있는데요,
ScoreCalculator가 int만 계산하고 Hand가 Score를 생성하는 현재 구조와, ScoreCalculator가 Score까지 반환하는 구조의 핵심 차이를 어떻게 보시는지 궁금합니다.
Score의 생성 위치가 다르긴 하지만, 사용하는 쪽에서는 결국 Hand.getScore()로 가져다 쓰게 되어서, 제가 느끼기엔 비슷한 방식이라고 생각되는데, 정답이 있는걸까요? 제이미의 의견이 궁금합니다.

이외에도 부족한 부분은 자유롭게 피드백 주시면 감사하겠습니다!